manager.js ➔ ... ➔ ??? - Code Metrics - Inspection of "fix:update was not working properly" - AdFabConnect/abejs - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a00f3d...6cfb01 )

by greg

created 2016-11-02 21:48 UTC

manager.js ➔ ... ➔ ??? A

↳ Parent: manager.js ➔ ... ➔ it(ꞌupdatePostInList() with new postꞌ)

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
dl	0
loc	7
rs	9.4285
c	0
b	0
f	0
nop	1

var chai = require('chai');
var path = require('path');

var Manager = require('../src/cli').Manager
var config = require('../src/cli').config
var cmsOperations = require('../src/cli').cmsOperations
config.set({root: __dirname + '/fixtures/'})


var cmsData = require('../src/cli').cmsData;
var Manager = require('../src/cli').Manager;

var fse = require('fs-extra');

describe('Manager', function() {
  before( function(done) {
    Manager.instance.init()
      .then(function () {

        this.fixture = {
          tag: fse.readFileSync(__dirname + '/fixtures/templates/article.html', 'utf8'),

          jsonArticle: fse.readJsonSync(__dirname + '/fixtures/files/article-4.json'),

          jsonArticle1: fse.readJsonSync(__dirname + '/fixtures/data/article-1.json')

        }
        done()
        
      }.bind(this))
  });

  it('getStructureAndTemplates()', function() {
    const data = Manager.instance.getStructureAndTemplates()
    chai.assert.equal(data['templates'][0].name, 'article-each-abe', 'failed !')
    chai.assert.equal(data['templates'].length, 7, 'failed !')
  });

  it('updateStructureAndTemplates()', function() {
    Manager.instance.updateStructureAndTemplates()
    const data = Manager.instance.getStructureAndTemplates()
    chai.assert.equal(data['templates'][0].name, 'article-each-abe', 'failed !')
    chai.assert.equal(data['templates'].length, 7, 'failed !')
  });

  it('getList()', function() {
    const list = Manager.instance.getList()
    chai.assert.equal(list[0].name, 'article-1.json', 'failed !')
    chai.assert.equal(list.length, 3, 'failed !')
  });

  it('getListWithStatusOnFolder() status publish', function() {
    const list = Manager.instance.getListWithStatusOnFolder('publish')
    chai.assert.equal(list[0].name, 'article-1.json', 'failed !')
    chai.assert.equal(list.length, 2, 'failed !')
  });

  it('getListWithStatusOnFolder() status draft', function() {
    const list = Manager.instance.getListWithStatusOnFolder('draft')
    chai.assert.equal(list.length, 2, 'failed !')
  });

  it('getListWithStatusOnFolder() status review', function() {
    const list = Manager.instance.getListWithStatusOnFolder('review')
    chai.assert.equal(list.length, 0, 'failed !')
  });

  it('getListWithStatusOnFolder() status publish /0-1', function() {
    const list = Manager.instance.getListWithStatusOnFolder('draft', '0-1')
    chai.assert.equal(list[0].name, 'article-2.json', 'failed !')
    chai.assert.equal(list.length, 1, 'failed !')
  });

  it('updatePostInList() with new post', function(done) {
    const len = Manager.instance.getList().length
    cmsOperations.create('article', '', 'article-4.html', {query: ''}, this.fixture.jsonArticle, false)
      .then(function(resSave) {
        const json = path.join(config.root, config.data.url, resSave.abe_meta.latest.abeUrl.replace('.html', '.json'))
        const list = Manager.instance.getList()
        fse.removeSync(json)
        chai.assert.equal(list.length, len + 1, 'failed !')
        done()
      }.bind(this));

  });
});


1			var chai = require('chai');
2			var path = require('path');
3
4			var Manager = require('../src/cli').Manager
5			var config = require('../src/cli').config
6			var cmsOperations = require('../src/cli').cmsOperations
7			config.set({root: __dirname + '/fixtures/'})
			0 ignored issues – show Compatibility introduced 2016-10-31 14:11 UTC by Report Bug Copy Issue Report Consider using the `path` module for constructing paths since they are otherwise not cross-OS compatible. Loading history...
8
9			var cmsData = require('../src/cli').cmsData;
10			var Manager = require('../src/cli').Manager;
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2016-10-31 14:11 UTC by Report Bug Copy Issue Report The variable `Manager` already seems to be declared on line `4`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
11			var fse = require('fs-extra');
12
13			describe('Manager', function() {
14			before( function(done) {
15			Manager.instance.init()
16			.then(function () {
17
18			this.fixture = {
19			tag: fse.readFileSync(__dirname + '/fixtures/templates/article.html', 'utf8'),
			0 ignored issues – show Compatibility introduced 2016-10-31 14:11 UTC by Report Bug Copy Issue Report Consider using the `path` module for constructing paths since they are otherwise not cross-OS compatible. Loading history...
20			jsonArticle: fse.readJsonSync(__dirname + '/fixtures/files/article-4.json'),
			0 ignored issues – show Compatibility introduced 2016-10-31 14:11 UTC by Report Bug Copy Issue Report Consider using the `path` module for constructing paths since they are otherwise not cross-OS compatible. Loading history...
21			jsonArticle1: fse.readJsonSync(__dirname + '/fixtures/data/article-1.json')
			0 ignored issues – show Compatibility introduced 2016-11-02 21:50 UTC by Report Bug Copy Issue Report Consider using the `path` module for constructing paths since they are otherwise not cross-OS compatible. Loading history...
22			}
23			done()
24
25			}.bind(this))
26			});
27
28			it('getStructureAndTemplates()', function() {
29			const data = Manager.instance.getStructureAndTemplates()
30			chai.assert.equal(data['templates'][0].name, 'article-each-abe', 'failed !')
31			chai.assert.equal(data['templates'].length, 7, 'failed !')
32			});
33
34			it('updateStructureAndTemplates()', function() {
35			Manager.instance.updateStructureAndTemplates()
36			const data = Manager.instance.getStructureAndTemplates()
37			chai.assert.equal(data['templates'][0].name, 'article-each-abe', 'failed !')
38			chai.assert.equal(data['templates'].length, 7, 'failed !')
39			});
40
41			it('getList()', function() {
42			const list = Manager.instance.getList()
43			chai.assert.equal(list[0].name, 'article-1.json', 'failed !')
44			chai.assert.equal(list.length, 3, 'failed !')
45			});
46
47			it('getListWithStatusOnFolder() status publish', function() {
48			const list = Manager.instance.getListWithStatusOnFolder('publish')
49			chai.assert.equal(list[0].name, 'article-1.json', 'failed !')
50			chai.assert.equal(list.length, 2, 'failed !')
51			});
52
53			it('getListWithStatusOnFolder() status draft', function() {
54			const list = Manager.instance.getListWithStatusOnFolder('draft')
55			chai.assert.equal(list.length, 2, 'failed !')
56			});
57
58			it('getListWithStatusOnFolder() status review', function() {
59			const list = Manager.instance.getListWithStatusOnFolder('review')
60			chai.assert.equal(list.length, 0, 'failed !')
61			});
62
63			it('getListWithStatusOnFolder() status publish /0-1', function() {
64			const list = Manager.instance.getListWithStatusOnFolder('draft', '0-1')
65			chai.assert.equal(list[0].name, 'article-2.json', 'failed !')
66			chai.assert.equal(list.length, 1, 'failed !')
67			});
68
69			it('updatePostInList() with new post', function(done) {
70			const len = Manager.instance.getList().length
71			cmsOperations.create('article', '', 'article-4.html', {query: ''}, this.fixture.jsonArticle, false)
72			.then(function(resSave) {
73			const json = path.join(config.root, config.data.url, resSave.abe_meta.latest.abeUrl.replace('.html', '.json'))
74			const list = Manager.instance.getList()
75			fse.removeSync(json)
76			chai.assert.equal(list.length, len + 1, 'failed !')
77			done()
78			}.bind(this));
			0 ignored issues – show unused-code introduced 2016-10-31 14:11 UTC by Report Bug Copy Issue Report The call to `bind` does not seem necessary since the function does not use `this`. Consider calling it directly. Loading history...
79			});
80			});
81

AdFabConnect / abejs

Push — master ( a00f3d...6cfb01 )

manager.js ➔ ... ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like